Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added ArchLinux package support #26

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

sarbesh
Copy link

@sarbesh sarbesh commented Dec 12, 2020

Archlinux package based on

  1. Release (Stable)
  2. Git commit (Development)

@rushabh-v
Copy link
Owner

Thanks for your contribution!

# The following guidelines are specific to BZR, GIT, HG and SVN packages.
# Other VCS sources are not natively supported by makepkg yet.

# Maintainer: Your Name <youremail@domain.com>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, put your name and email here!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done


FILE=/usr/share/pam-configs/Facerec
SUDO=/etc/pam.d/sudo
TEXT='auth sufficient pam_python.so /lib/Auth/Facerec/pam.py'
Copy link
Owner

@rushabh-v rushabh-v Dec 12, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this won't work. The file will be /usr/lib/Auth/Facerec/pam_ptn.py

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/usr/lib is symlink to /lib

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but the file name is pam_ptn.py

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

renamed

if grep -q $TEXT $SUDO;then
sed -i $TEXT $SUDO
else
echo "NotE"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a typo!?

Copy link
Author

@sarbesh sarbesh Dec 19, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have removed the block, as it was not functing as expected during installation, instead user has to manually ass the following line auth sufficient pam_python.so /lib/Auth/Facerec/pam_ptn.py to /etc/pam.d/sudo to enable and disable the authentication package

Comment on lines +1 to +10
import sys
import numpy as np

from os import system

if __name__ == '__main__':

sp = np.array(sys.path)
np.save("/usr/lib/Auth/Facerec/deps_path.npy", sp)
system("sudo python3 /usr/lib/Auth/Facerec/cli_info.py")
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't we use the build.py that is already there?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do not need common-auth for ArchLinux instead, it should be done manually by adding or removing auth sufficient pam_python.so /lib/Auth/Facerec/pam_ptn.py from /etc/pam.d/sudo to enable and disable respectively.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants